Skip to content

Conversation

baronfel
Copy link
Member

@baronfel baronfel commented Oct 1, 2025

This is a the result of merging release/10.0.1xx-rc2 and release/10.0.1xx, and keeping just the code changes. Dependency flows are more complex and I don't have the domain knowledge to deal with them.

@baronfel baronfel requested a review from nagilson October 1, 2025 21:28
Copy link
Member

@nagilson nagilson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - I'm not sure why release/10.0.1xx...release/10.0.1xx-rc2#diff-58f51f11a7ac3d6d7da0423eeaeef2d82bfd47b27d7663ec9a842acb07b16487 is wrong about some of the missing code changes, even the first line (changing {RuntimeIdentifier} to (RuntimeIdentifier) when that exact sha isn't in the history but the code change is there in 10.0... GitHub diffs are just wrong sometimes.

Anyways, I verified some of the other missing changes and this seems good. Thank you.

@nagilson nagilson enabled auto-merge October 1, 2025 21:33
@nagilson nagilson merged commit d9308af into dotnet:release/10.0.1xx Oct 1, 2025
26 of 27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants